feat(modules): promote xlsx to builtin module#86
Merged
simongdavies merged 2 commits intohyperlight-dev:mainfrom Apr 28, 2026
Merged
feat(modules): promote xlsx to builtin module#86simongdavies merged 2 commits intohyperlight-dev:mainfrom
simongdavies merged 2 commits intohyperlight-dev:mainfrom
Conversation
Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new Hyperlight builtin module (ha:xlsx) for generating Excel .xlsx workbooks, and updates skill/docs/tests to support and validate usage (including standardizing skill allowed-tools metadata).
Changes:
- Introduces
builtin-modules/src/xlsx.tsplus module metadata/config/types to exposeha:xlsx. - Adds sandbox integration tests that smoke-test XLSX generation and validate emitted OOXML/ZIP contents.
- Standardizes/validates skill
allowed-toolsmetadata (and updates multiple skill guides to referenceha:xlsxwhere relevant).
Reviewed changes
Copilot reviewed 17 out of 18 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/sandbox-tool.test.ts | Adds ZIP parsing helpers + XLSX module end-to-end sandbox tests. |
| tests/pattern-integrity.test.ts | Validates skill allowed-tools entries against ALLOWED_TOOLS and enforces MCP tool presence. |
| skills/xlsx-expert/SKILL.md | New skill guidance for building .xlsx with ha:xlsx. |
| skills/web-scraper/SKILL.md | Adds allowed-tools metadata block. |
| skills/research-synthesiser/SKILL.md | Adds allowed-tools metadata block + formatting tweaks. |
| skills/report-builder/SKILL.md | Adds allowed-tools metadata block + references ha:xlsx for binary reports. |
| skills/pptx-expert/SKILL.md | Expands allowed-tools list + minor formatting tweaks. |
| skills/pdf-expert/SKILL.md | Expands allowed-tools list + Markdown table formatting adjustments. |
| skills/mcp-services/SKILL.md | Expands allowed-tools list + minor formatting tweaks. |
| skills/data-processor/SKILL.md | Adds allowed-tools metadata block + mentions ha:xlsx as an available module. |
| skills/api-explorer/SKILL.md | Adds allowed-tools metadata block + improves table formatting. |
| builtin-modules/zip-format.json | Adds ha:xlsx to related modules list. |
| builtin-modules/xml-escape.json | Notes XLSX as a consumer of ha:xml-escape. |
| builtin-modules/xlsx.json | New module metadata for xlsx. |
| builtin-modules/tsconfig.json | Adds TS path mapping for ha:xlsx. |
| builtin-modules/src/xlsx.ts | New XLSX builder implementation (workbook/sheet/styles/charts/pivots/etc.). |
| builtin-modules/src/types/ha-modules.d.ts | Adds ambient TS declarations for ha:xlsx. |
| builtin-modules/CLAUDE.md | Updates module list table to include xlsx. |
Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces the new
ha:xlsxmodule for Excel workbook generation and updates documentation and skill guidance to support its use. It also standardizes allowed tool lists across multiple skills, enhances documentation formatting, and clarifies best practices and usage patterns for report building and document generation.New module and integration:
ha:xlsxmodule for creating Excel XLSX workbooks, with comprehensive metadata, usage patterns, critical rules, and anti-patterns. This includes updates to module listings, documentation tables, and TypeScript config to enable importingha:xlsx.ha:xml-escapeandha:zip-formatto referenceha:xlsxas a consumer/peer.Skill documentation improvements:
ha:xlsxas a recommended module for tabular data export in the Data Processor and Report Builder skills, with guidance for binary output and profile usage.Standardization of allowed tools:
allowed-toolslists across skills including API Explorer, Data Processor, MCP Services, PDF Expert, PPTX Expert, Report Builder, and Research Synthesiser for consistent sandbox and plugin capabilities.Documentation formatting and workflow clarifications:
General enhancements: